pull request

自分がどう動作確認しているかを書きませう

コードだけでなく、動作確認方法もレビューしてもらひませう

動作確認方法がレビューできれば、手元で動かさなくてもLGTMできる、かもしれない

pull requestのレビューというのは、責任の分散の爲のダブルチェックなのだから

まずチェック項目に漏れが無い事を確認するべき

設定畫面を變更したけど、これログイン畫面にも影響しているよね、とか

次にチェック項目を 2 人とも實行してみて、本當に ok か確認する

元々責任の分散の爲の風習なので、輕くやるべきなのは確かなのだが

確實に責任分散する爲に

確實にダブルチェックしましょう

動作確認方法が明示されていないと、どちらか一方に責任が偏ってしまう

code:每日のレビュータイム

これはがんばらないといけませんね (PullRequest がなかなかレビューされないから存在してゐるのだと思ふので)。前ゐたリアルタイムサーバーチームでは、全員が全 PullRequest を見るのを當然にゐたので (レビューを merge の條件にしない、merge 後でもレビューする)。ただし「レビューは質を擔保するものではなく會話の機會である (「レビュアーはレビュイーより詳しい」と云ふ前提を否定する)」と云ふ考へでやってゐたので、どう考へてゆくかは不明。